Skip to content

fix(ci): remove flake-checker-action (Node.js 20 deprecation)#95

Merged
JacobPEvans merged 1 commit intomainfrom
bugfix/node20-deprecation
Mar 15, 2026
Merged

fix(ci): remove flake-checker-action (Node.js 20 deprecation)#95
JacobPEvans merged 1 commit intomainfrom
bugfix/node20-deprecation

Conversation

@JacobPEvans
Copy link
Owner

Summary

  • Removes DeterminateSystems/flake-checker-action@main from _nix-validate.yml reusable workflow
  • determinate-nix-action@v3 (already in use for Nix installation) bundles flake checking, making the separate step redundant
  • Eliminates the Node.js 20 deprecation warning ahead of GitHub's June 2, 2026 deadline

Test plan

  • CI runs without Node.js 20 deprecation warnings
  • nix flake check still passes in downstream repos that consume this reusable workflow

🤖 Generated with Claude Code

…determinate-nix-action@v3

Removes DeterminateSystems/flake-checker-action which runs on Node.js 20,
deprecated on GitHub Actions June 2, 2026. determinate-nix-action@v3 is the
consolidated replacement that bundles both Nix installation and flake checking.

(claude)
Copilot AI review requested due to automatic review settings March 15, 2026 06:09
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the DeterminateSystems/flake-checker-action@main step from the _nix-validate.yml reusable workflow to eliminate a Node.js 20 deprecation warning, since determinate-nix-action@v3 already bundles flake checking.

Changes:

  • Removed the "Lint flake.lock" step that used DeterminateSystems/flake-checker-action@main

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@JacobPEvans JacobPEvans merged commit 4f25b59 into main Mar 15, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants